CreateLeaveRequestCommandHandler.execute - Code Metrics - Inspection of "feat: mattermost notifier" - fairnesscoop/permacoop - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#436)

by Mathieu

created 2024-05-31 12:34 UTC

CreateLeaveRequestCommandHandler.execute B

↳ Parent: CreateLeaveRequestCommandHandler

Complexity

Conditions

Size

Total Lines	55
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	37
dl	0
loc	55
rs	8.9919
c	0
b	0
f	0
cc	3

How to fix Long Method

import { Inject } from '@nestjs/common';
import { CommandHandler } from '@nestjs/cqrs';
import { CreateLeaveRequestCommand } from './CreateLeaveRequestCommand';
import { ILeaveRequestRepository } from 'src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository';
import { LeaveRequest } from 'src/Domain/HumanResource/Leave/LeaveRequest.entity';
import { DoesLeaveRequestExistForPeriod } from 'src/Domain/HumanResource/Leave/Specification/DoesLeaveRequestExistForPeriod';
import { LeaveRequestAlreadyExistForThisPeriodException } from 'src/Domain/HumanResource/Leave/Exception/LeaveRequestAlreadyExistForThisPeriodException';
import { DoesLeaveExistForPeriod } from 'src/Domain/FairCalendar/Specification/DoesLeaveExistForPeriod';
import { EventsOrLeavesAlreadyExistForThisPeriodException } from 'src/Domain/FairCalendar/Exception/EventsOrLeavesAlreadyExistForThisPeriodException';
import { IMattermostNotifier } from 'src/Application/IMattermostNotifier';
import { ICommandBus } from 'src/Application/ICommandBus';
import { CreateNotificationCommand } from 'src/Application/Notification/Command/CreateNotificationCommand';
import { NotificationType } from 'src/Domain/Notification/Notification.entity';

@CommandHandler(CreateLeaveRequestCommand)
export class CreateLeaveRequestCommandHandler {
  constructor(
    @Inject('ILeaveRequestRepository')
    private readonly leaveRequestRepository: ILeaveRequestRepository,
    @Inject('ICommandBus')
    private readonly commandBus: ICommandBus,
    private readonly doesLeaveRequestExistForPeriod: DoesLeaveRequestExistForPeriod,
    private readonly doesLeaveExistForPeriod: DoesLeaveExistForPeriod,
  ) {}

  public async execute(command: CreateLeaveRequestCommand): Promise<string> {
    const {
      user,
      endDate,
      endsAllDay,
      type,
      startDate,
      startsAllDay,
      comment
    } = command;

    if (
      true ===
      (await this.doesLeaveRequestExistForPeriod.isSatisfiedBy(
        user,
        startDate,
        endDate
      ))
    ) {
      throw new LeaveRequestAlreadyExistForThisPeriodException();
    }

    if (
      true ===
      (await this.doesLeaveExistForPeriod.isSatisfiedBy(
        user,
        startDate,
        endDate
      ))
    ) {
      throw new EventsOrLeavesAlreadyExistForThisPeriodException();
    }

    const leaveRequest = await this.leaveRequestRepository.save(
      new LeaveRequest(
        user,
        type,
        startDate,
        startsAllDay,
        endDate,
        endsAllDay,
        comment
      )
    );

    this.commandBus.execute(
      new CreateNotificationCommand(
        NotificationType.POST,
        'Demande de congés',
        leaveRequest
      )
    );

    return leaveRequest.getId();
  }
}


1			import { Inject } from '@nestjs/common';
2			import { CommandHandler } from '@nestjs/cqrs';
3			import { CreateLeaveRequestCommand } from './CreateLeaveRequestCommand';
4			import { ILeaveRequestRepository } from 'src/Domain/HumanResource/Leave/Repository/ILeaveRequestRepository';
5			import { LeaveRequest } from 'src/Domain/HumanResource/Leave/LeaveRequest.entity';
6			import { DoesLeaveRequestExistForPeriod } from 'src/Domain/HumanResource/Leave/Specification/DoesLeaveRequestExistForPeriod';
7			import { LeaveRequestAlreadyExistForThisPeriodException } from 'src/Domain/HumanResource/Leave/Exception/LeaveRequestAlreadyExistForThisPeriodException';
8			import { DoesLeaveExistForPeriod } from 'src/Domain/FairCalendar/Specification/DoesLeaveExistForPeriod';
9			import { EventsOrLeavesAlreadyExistForThisPeriodException } from 'src/Domain/FairCalendar/Exception/EventsOrLeavesAlreadyExistForThisPeriodException';
10			import { IMattermostNotifier } from 'src/Application/IMattermostNotifier';
11			import { ICommandBus } from 'src/Application/ICommandBus';
12			import { CreateNotificationCommand } from 'src/Application/Notification/Command/CreateNotificationCommand';
13			import { NotificationType } from 'src/Domain/Notification/Notification.entity';
14
15			@CommandHandler(CreateLeaveRequestCommand)
16			export class CreateLeaveRequestCommandHandler {
17			constructor(
18			@Inject('ILeaveRequestRepository')
19			private readonly leaveRequestRepository: ILeaveRequestRepository,
20			@Inject('ICommandBus')
21			private readonly commandBus: ICommandBus,
22			private readonly doesLeaveRequestExistForPeriod: DoesLeaveRequestExistForPeriod,
23			private readonly doesLeaveExistForPeriod: DoesLeaveExistForPeriod,
24			) {}
25
26			public async execute(command: CreateLeaveRequestCommand): Promise<string> {
27			const {
28			user,
29			endDate,
30			endsAllDay,
31			type,
32			startDate,
33			startsAllDay,
34			comment
35			} = command;
36
37			if (
38			true ===
39			(await this.doesLeaveRequestExistForPeriod.isSatisfiedBy(
40			user,
41			startDate,
42			endDate
43			))
44			) {
45			throw new LeaveRequestAlreadyExistForThisPeriodException();
46			}
47
48			if (
49			true ===
50			(await this.doesLeaveExistForPeriod.isSatisfiedBy(
51			user,
52			startDate,
53			endDate
54			))
55			) {
56			throw new EventsOrLeavesAlreadyExistForThisPeriodException();
57			}
58
59			const leaveRequest = await this.leaveRequestRepository.save(
60			new LeaveRequest(
61			user,
62			type,
63			startDate,
64			startsAllDay,
65			endDate,
66			endsAllDay,
67			comment
68			)
69			);
70
71			this.commandBus.execute(
72			new CreateNotificationCommand(
73			NotificationType.POST,
74			'Demande de congés',
75			leaveRequest
76			)
77			);
78
79			return leaveRequest.getId();
80			}
81			}
82

fairnesscoop / permacoop

Pull Request — master (#436)

CreateLeaveRequestCommandHandler.execute B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like